Conversation
|
Maybe adding a feature config option for this? And use it for the migration, there is also the model helpers that use |
|
I'm not sure about this because |
|
I also agree on this, it's simpler and more efficient to use |
|
My opinion is it's better for the project to be quite flexible considering this feature not that hard to maintain. If anybody prefer to use uuid as a primary key, why not? |
|
Yes, I agree, we can add it as an optional feature. |
|
Hi @yamaha252 I'm ready to accept and merge this but there are more tables that use integer columns for id references that would need to be changed ( |
|
Thanks @ifox, done |
|
Yes, I think that's good, as it'll supprt both UUID and ULID. |
# Conflicts: # src/Http/Controllers/Admin/ModuleController.php
Hi!
As I can see there is no big reason not to support model uuid primary keys.
It's always annoying I have to use autoincrement int if I want to use the model in Twill.